Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

sql: rename anonymizedStmt in sqlstats pkg to stmtNoConstants #83428

Merged
merged 1 commit into from
Jun 28, 2022
Merged

sql: rename anonymizedStmt in sqlstats pkg to stmtNoConstants #83428

merged 1 commit into from
Jun 28, 2022

Conversation

ericharmeling
Copy link
Contributor

Note that this commit does not change any files outside the sqlstats package.

Fixes #80725.

Release note: None

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@ericharmeling ericharmeling force-pushed the rename-anonymizedStmt branch from d402312 to e1b4bfb Compare June 27, 2022 18:29
@ericharmeling
sql: rename anonymizedStmt in sqlstats pkg to stmtNoConstants
c9267e2 
This commit changes all instances of anonymizedStmt to stmtNoConstants in
the sqlstats package.

The commit does not change any files outside the sqlstats package.

Fixes cockroachdb#80725.

Release note: None
@ericharmeling ericharmeling force-pushed the rename-anonymizedStmt branch from e1b4bfb to c9267e2 Compare June 28, 2022 12:45
@ericharmeling ericharmeling requested a review from a team June 28, 2022 14:42
maryliag
maryliag approved these changes Jun 28, 2022
View reviewed changes
Copy link
Contributor

@maryliag maryliag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 4 of 4 files at r1, all commit messages.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @ericharmeling)

@ericharmeling
Copy link
Contributor Author

bors r+

@craig
Copy link
Contributor

craig bot commented Jun 28, 2022

Build succeeded:

@craig craig bot merged commit 5541cf8 into cockroachdb:master Jun 28, 2022
@blathers-crl blathers-crl bot mentioned this pull request Jun 28, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maryliag maryliag maryliag approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

sql: rename anonymizedStmt in sqlstats pkg to stmtNoConstants
3 participants
@ericharmeling @cockroach-teamcity @maryliag